Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added two new screenshots to curiefense integration #5038

Closed
wants to merge 1 commit into from
Closed

added two new screenshots to curiefense integration #5038

wants to merge 1 commit into from

Conversation

thakuraman24
Copy link

@thakuraman24 thakuraman24 commented Oct 13, 2023

Description

This PR fixes #4884

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Oct 13, 2023

🚀 Preview for commit e4d08eb at: https://652960322e23ff06288da114--layer5.netlify.app

Copy link
Contributor

@singh1203 singh1203 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thakuraman24 Does your screenshot represent the Curiefense Integration.?
Also, we need at least four screenshots with the specific integration.
Take this as reference https://github.com/layer5io/layer5/pull/4990 and Meshmap-designer Walkthrough
Thank you for your contribution.

@thakuraman24
Copy link
Author

@singh1203 I am having two problems while contributing to this issue

-->The first one is that I am totally new to this cloud-native field and don't know how to design these Meshmaps of different integrations, so can you provide me with some references that I can use to create a CURIEFENSE SPECIFIC Meshmap?

-->The second one is that the LIGHTHOSECI test for the above PR failed do you have any idea what could be the reason for that?

@singh1203
Copy link
Contributor

@singh1203 I am having two problems while contributing to this issue

-->The first one is that I am totally new to this cloud-native field and don't know how to design these Meshmaps of different integrations, so can you provide me with some references that I can use to create a CURIEFENSE SPECIFIC Meshmap?

-->The second one is that the LIGHTHOSECI test for the above PR failed do you have any idea what could be the reason for that?

Thank you for pointing it out, feel free to open the thread in slack or discussion forum on Layer5 discuss forum where me along with other member's will help you out.

Copy link
Member

@Yashsharma1911 Yashsharma1911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thakuraman24 thankyou for working on it, make sure these acceptance test follows, and mark them as completed, also regarding design you don't need to create your own, you can use published design and clone them and take screenshot of them (You can do that in meshmap)

  • Ensure your screenshot represents the use of a few specific integrations.
  • The screenshot from the Designer mode should contain a valid design, refer to meshery catalog to use and view published designs.
  • Add both Dark Mode and Light Mode screenshots.
  • The screenshot should not be a snippet, take the full screenshot of your browser.
  • Use Incognito mode on Chrome browser, to ensure your bookmark bar or any other items are not shown in the screenshot/video.

@saurabh100ni
Copy link
Contributor

Thanks for raising PR, Let's discuss this in the Website call today at 5:30 PM IST.
Please add this as your agenda in Docs

@thakuraman24 thakuraman24 closed this by deleting the head repository Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[Screenshots] Curiefense under 'How it Works See It in Action' section
5 participants